Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: smooth loop + draggable carousel #496

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

r3yc0n1c
Copy link
Contributor

Description

  • fixed the Smooth Loop transition for venue cards
  • added Draggable Carousel for venue cards

Related issue(s)
Fixes #490

Demo

draggable-carousal.mp4

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for peaceful-ramanujan-288045 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 66ef649
🔍 Latest deploy log https://app.netlify.com/sites/peaceful-ramanujan-288045/deploys/6761c04714e2370008c4af2c
😎 Deploy Preview https://deploy-preview-496--peaceful-ramanujan-288045.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@AceTheCreator AceTheCreator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really awesome!

Do you think you can decrease the spacing between the venues cards. I think making them closer in mobile view will be aesthetically pleasing. wyt?

cc @ashmit-coder your thought will be appreciated here

@r3yc0n1c
Copy link
Contributor Author

This looks really awesome!

Do you think you can decrease the spacing between the venues cards. I think making them closer in mobile view will be aesthetically pleasing. wyt?

cc @ashmit-coder your thought will be appreciated here

Good idea! Yes, we can do that. Actually, I tried to keep the previous layout. Let me know what others think about the spacing. In the meantime, I'll continue working on the other issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Smooth loop for venue cards slider + draggable carousel
2 participants